Skip to content

docs: document ALLOW_INSECURE_GIT_ACCESS for trusted local git providers#521

Merged
VascoSch92 merged 2 commits into
OpenHands:mainfrom
abhikb2005:codex/docs-allow-insecure-git-access
May 30, 2026
Merged

docs: document ALLOW_INSECURE_GIT_ACCESS for trusted local git providers#521
VascoSch92 merged 2 commits into
OpenHands:mainfrom
abhikb2005:codex/docs-allow-insecure-git-access

Conversation

@abhikb2005
Copy link
Copy Markdown
Contributor

Summary

Adds documentation for ALLOW_INSECURE_GIT_ACCESS when using trusted local/internal Gitea or Forgejo instances over HTTP.

Context

Related to OpenHands/OpenHands#14523.

@VascoSch92, you asked me to tag you in the PR.

Scope

This PR is intentionally docs-only.

It:

  • Adds ALLOW_INSECURE_GIT_ACCESS to the environment variables reference.
  • Clarifies that it is intended only for trusted local/internal HTTP git providers.
  • Mentions local Gitea/Forgejo-style usage.

It does not:

  • Change runtime behavior.
  • Claim to fully fix the environment propagation behavior described in #14523.
  • Modify unrelated documentation.

Validation

  • Confirmed source behavior references ALLOW_INSECURE_GIT_ACCESS.
  • Confirmed git diff --check passes.
  • Ran the narrowed relevant pytest target: 30 passed, 6 deselected.
  • Full pytest currently has an unrelated upstream failure caused by a raw GitHub URL returning 404.
  • Mintlify CLI validation was attempted but unavailable/timed out via npx.

Copy link
Copy Markdown
Member

@VascoSch92 VascoSch92 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a small comment.

Otherwise, LGTM

Comment thread openhands/usage/environment-variables.mdx Outdated
@abhikb2005 abhikb2005 force-pushed the codex/docs-allow-insecure-git-access branch from 066df50 to 33cca94 Compare May 28, 2026 06:33
@abhikb2005
Copy link
Copy Markdown
Contributor Author

abhikb2005 commented May 28, 2026 via email

@VascoSch92 VascoSch92 requested a review from enyst May 28, 2026 06:48
@VascoSch92
Copy link
Copy Markdown
Member

@abhikb2005 thanks :-)

@VascoSch92 VascoSch92 merged commit 563cafb into OpenHands:main May 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants